Skip to content

Retrieval API implementation#1744

Open
mgwojciech wants to merge 1 commit intodevfrom
feature/1741-retrieval-api
Open

Retrieval API implementation#1744
mgwojciech wants to merge 1 commit intodevfrom
feature/1741-retrieval-api

Conversation

@mgwojciech
Copy link
Copy Markdown

Added support for using MS Graph API Copilot Retrieval API
#1741

@pkbullock
Copy link
Copy Markdown
Member

Hey @mgwojciech, can you expand to include the docs for this please? To give examples of how to use and permissions, id use other areas as an example of documentation standard.

@mgwojciech
Copy link
Copy Markdown
Author

Sure thing. I wanted to verify first if everything is technically sounds. If that the case I will add the docs

@pkbullock
Copy link
Copy Markdown
Member

pkbullock commented Apr 4, 2026

Awesome stuff! I have been testing this.

For the retrieval API, this looks good and can retrieve results (once I remembered to setup the tests lol).

The test when in non-mocking setup, would only work though for your setup (mocking will be fine of course), just wondering if you should add comments or code to setup the test for the next person working on this can ensure it passes when not mocking.

There is a pattern of sync/async methods in other endpoints we have covered e.g. SharePoint, wondering if this too should align too?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants